-
Notifications
You must be signed in to change notification settings - Fork 193
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: support overloaded system functions in system libraries #3397
base: vdrg/fix-library-imports
Are you sure you want to change the base?
Conversation
|
6aaed39
to
007be2b
Compare
encodeCall
was unable to differentiate between overloaded system functions, so i needed to manually construct the selector withencodeWithSelector
.i had to add the ability for codegen to parse struct definitions from source files in order to generate full function signatures. this adds one restriction when writing systems: you can't import types from the generated libraries themselves. if this is done, when generating for the first time the system libraries won't be available and it will fail when attempting to find type definitions.
i also punted on being able to import structs from non-relative paths and use them in function signatures. i didn't need it and it was a bit more complicated, but lmk if you think that is important to add.